Skip to content

Fix issue #237 - Incorrect usage of return value in flash_file method#263

Merged
hkpeprah merged 2 commits intosquare:masterfrom
TheColector:fix/ISSUE-237/fix_flash_file_method_return
Mar 13, 2026
Merged

Fix issue #237 - Incorrect usage of return value in flash_file method#263
hkpeprah merged 2 commits intosquare:masterfrom
TheColector:fix/ISSUE-237/fix_flash_file_method_return

Conversation

@TheColector
Copy link
Copy Markdown
Contributor

This PR fixes the issue mentioned by me in #237

JLINK_DownloadFile from JLINK's DLL does not return number of bytes flashed.
It returns error code
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 10, 2026

CLA assistant check
All committers have signed the CLA.

@TheColector TheColector changed the title Fix/issue 237/fix flash file method return Fix issue #237 - Incorrect usage of return value in flash_file method Mar 10, 2026
@hkpeprah
Copy link
Copy Markdown
Contributor

LGTM. You just have to sign the CLA for us to accept this change though.

@TheColector
Copy link
Copy Markdown
Contributor Author

It's already been signed on my end.
I figure it got stuck somehow as some people described the same issue had happened to them as well.
I'll try to push empty commit to retrigger checks

@TheColector TheColector force-pushed the fix/ISSUE-237/fix_flash_file_method_return branch from ffa6506 to 8900a85 Compare March 12, 2026 10:12
@hkpeprah hkpeprah closed this Mar 13, 2026
@hkpeprah hkpeprah reopened this Mar 13, 2026
@hkpeprah hkpeprah self-requested a review March 13, 2026 16:39
Copy link
Copy Markdown
Contributor

@hkpeprah hkpeprah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Thank you! 👍

@hkpeprah hkpeprah merged commit a2172aa into square:master Mar 13, 2026
6 checks passed
@hkpeprah
Copy link
Copy Markdown
Contributor

Will release this with the other change for the exec_command when its available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants